Skip to content

Planning reports #8631

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Planning reports #8631

wants to merge 3 commits into from

Conversation

plotnick
Copy link
Contributor

@plotnick plotnick commented Jul 17, 2025

Fixes #8284, fixes #8548, maybe (when tested) #8285, #8298.

Replaces many (but not yet all) planner log messages with entries in a structured planning report. Those reports are comprised of "step reports", each of which corresponds to a subroutine of do_plan. The complete report may be formatted (in English) via a Display implementation. If planning from the blueprint_planner background task is successful, a report is attached to the BlueprintPlannerStatus::Targeted variant so that it may be displayed via omdb.

TODO:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

planner: adding new zones needs to know whether all lower-level updates are complete planner needs way to report events that it's waiting for
1 participant